From 5566034ecd3c89e686de4d9231694d7074ea8ae4 Mon Sep 17 00:00:00 2001
From: robertlipe <robertlipe@f51c46e8-681c-474f-0cfe-069cfd0219fb>
Date: Sun, 29 Mar 2015 05:21:39 +0000
Subject: [PATCH] Resist malformed Lowrance USR files.

---
 gpsbabel/csv_util.cc    |  3 +++
 gpsbabel/lowranceusr.cc | 11 ++++++-----
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/gpsbabel/csv_util.cc b/gpsbabel/csv_util.cc
index 3aeb2fc90..78ad8ed74 100644
--- a/gpsbabel/csv_util.cc
+++ b/gpsbabel/csv_util.cc
@@ -754,6 +754,9 @@ void xcsv_file_init(void)
 }
 
 XcsvFile::XcsvFile() {
+  is_internal = false;
+  ifield_ct = ofield_ct = 0;
+  extension = description = NULL;
 //   xcsv_file_init(); 
 }
 
diff --git a/gpsbabel/lowranceusr.cc b/gpsbabel/lowranceusr.cc
index 206d36ef2..c2197569e 100644
--- a/gpsbabel/lowranceusr.cc
+++ b/gpsbabel/lowranceusr.cc
@@ -505,7 +505,7 @@ lowranceusr_parse_icons(void)
     printf(MYNAME " parse_icons: num Icons = %d\n", num_icons);
   }
 
-  for (i=0; i < num_icons; i++) {
+  for (i=0; i < num_icons && !gbfeof(file_in); i++) {
     if (ignoreicons) {
       /* position coord lat & long */
       (void) gbfread(&buff[0], 4, 2, file_in);
@@ -546,7 +546,7 @@ lowranceusr_parse_trails(void)
     printf(MYNAME " parse_trails: num trails = %d\n", num_trails);
   }
 
-  for (i=trk_num=0; i < num_trails; i++) {
+  for (i=trk_num=0; i < num_trails && !gbfeof(file_in); i++) {
     trk_head = route_head_alloc();
     trk_head->rte_num = ++trk_num;
     track_add_head(trk_head);
@@ -585,7 +585,7 @@ lowranceusr_parse_trails(void)
 
     if (num_trail_points) {
 
-      while (num_trail_points) {
+      while (num_trail_points && !gbfeof(file_in)) {
         /* num section points */
         num_section_points = gbfgetint16(file_in);
 
@@ -593,7 +593,8 @@ lowranceusr_parse_trails(void)
           printf(MYNAME " parse_trails: num section points = %d\n", num_section_points);
         }
 
-        for (j=0; j < num_section_points; j++, num_trail_points--) {
+        for (j=0; j < num_section_points && !gbfeof(file_in);
+             j++, num_trail_points--) {
           wpt_tmp = new Waypoint;
           wpt_tmp->latitude = lat_mm_to_deg(gbfgetint32(file_in));
           wpt_tmp->longitude = lon_mm_to_deg(gbfgetint32(file_in));
@@ -652,7 +653,7 @@ data_read(void)
     printf(MYNAME " data_read: Num waypoints %d\n", NumWaypoints);
   }
 
-  for (i = 0; i < NumWaypoints; i++) {
+  for (i = 0; i < NumWaypoints && !gbfeof(file_in); i++) {
     Waypoint* wpt_tmp;
 
     wpt_tmp = new Waypoint;
-- 
2.30.2